Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

+ (add) Importer logic #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

+ (add) Importer logic #1

wants to merge 1 commit into from

Conversation

Apke
Copy link
Collaborator

@Apke Apke commented Oct 26, 2016

try tzo understand before saying something about speedness

@razbakov
Copy link

  • see also https://github.com/avstudnitz/AvS_FastSimpleImport, it seems to have the same idea and structure
  • rename package composer-name
  • remove .idea
  • add some introduction before config jsdoc in readme, so it is easier understand how to read it
  • check perfomance, because you use $model->save() and it is very slow solution, see alternative here:

AvS_FastSimpleImport updates updated field for the product
see https://github.com/avstudnitz/AvS_FastSimpleImport/blob/master/src/app/code/community/AvS/FastSimpleImport/Model/Import/Entity/Product.php#L1087

AvS_FastSimpleImport uses Mage_ImportExport_Model_Import_Entity_Product::_saveProductAttributes
see https://github.com/avstudnitz/AvS_FastSimpleImport/blob/04979b81f74b2fec242ec5bb05e204c733257112/src/app/code/community/AvS/FastSimpleImport/Model/Import/Entity/Product.php#L1225

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants